-
Notifications
You must be signed in to change notification settings - Fork 670
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: extended hpdc cache subsystem #1978
feat: extended hpdc cache subsystem #1978
Conversation
output logic icache_miss_ready_o, | ||
input logic icache_miss_valid_i, | ||
input hpdcache_mem_req_t icache_miss_i, | ||
input hpdcache_mem_id_t icache_miss_id_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input hpdcache_mem_id_t icache_miss_id_i, | |
input hpdcache_mem_id_t icache_miss_id_i, |
❌ failed run, report available here. |
Thanks for the contribution @takeshiho0531! I have left a few comments and will leave it to other members of the team to review/approve this PR. In addition to the specific comments, it would be very good indeed if you could provide a markdown (rst) file documenting how you have extend hpdcache so that it can be used as an I$. |
❌ failed run, report available here. |
❌ failed run, report available here. |
❌ failed run, report available here. |
❌ failed run, report available here. |
@takeshiho0531 THis PR is open for a while. What is his status? If not needed for cva6 project, please close it. |
WHAT
detail
axi arbiter
for extended hpdcHOW to extend hpdcache
cva6_hpdcache_subsystem.sv
by High Performance Instruction Cache